Skip to content

Check that symbol accesses are single threaded #954

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 10 commits into from
Closed

Check that symbol accesses are single threaded #954

wants to merge 10 commits into from

Conversation

odersky
Copy link
Contributor

@odersky odersky commented Nov 13, 2015

No description provided.

@odersky
Copy link
Contributor Author

odersky commented Nov 13, 2015

/rebuild

@propensive propensive changed the title Check that symol accesses are single threaded Check that symbol accesses are single threaded Nov 13, 2015
@odersky odersky closed this Nov 13, 2015
@odersky odersky reopened this Nov 13, 2015
@odersky
Copy link
Contributor Author

odersky commented Nov 17, 2015

I don't think we need this anymore.

@odersky odersky closed this Nov 17, 2015
OlivierBlanvillain pushed a commit to OlivierBlanvillain/dotty that referenced this pull request Dec 8, 2016
@odersky @OlivierBlanvillain
address "this would catch all throwables" warnings
7eecbfe 
original patch by @odersky in scala#955 -- criterion for the refactor:
"catch Throwable as long as there's no obvious control flow exception going through the catch
and the caught exception is processed further"

rebased & updated with review comments in scala#955 and scala#954
OlivierBlanvillain pushed a commit to OlivierBlanvillain/dotty that referenced this pull request Dec 12, 2016
@odersky @OlivierBlanvillain
address "this would catch all throwables" warnings
7d50958 
original patch by @odersky in scala#955 -- criterion for the refactor:
"catch Throwable as long as there's no obvious control flow exception going through the catch
and the caught exception is processed further"

rebased & updated with review comments in scala#955 and scala#954
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@odersky